-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Describe how to build a Docker image with an example #29047
Conversation
That's only related if you want to push an image you did not create in the first place. Maybe reword it to something like
And add it below the push command section because that should be part of a docker tutorial which the Gitea docs are not. |
Mhm okay. Actually I created that image myself. So not like I wanted to upload the Ubuntu latest or something |
1633a74
to
d5e5989
Compare
@lunny moved the part under the full push section and added it as in case of an error. |
I don't want to block the change but for me this is comparable to: "We have a command We already have a chapter https://docs.gitea.com/usage/packages/container#image-naming-convention which describes how the image name must be structured. |
@KN4CK3R the confusing thing for me as a relative basic user of docker was that for me the tag was „latest“ and I didn’t get automatically that I need to write a new tag where the i needed tongue the domain image and tag into it. My idea was to improve the handling for newer users. And I would have found that info in the doc helpful. But I also don’t mind if you don’t merge it. |
I think you read the docs and may or may not have noticed the chapter I linked. Could we improve that block, so you don't run into the problem? |
I saw the chapter but i did not consider it helpful for my issue. Because I wasn’t aware that the „name“ of an image is also called tag? For me the tag was always the :{tag} like latest. For example name of the image ubuntu and tag latest. So I think the double meaning of the tag got me confused. I thought the naming convention part was only relevant for the server so that there is a structure. Maybe I should have been more sceptical because actually I expected a cli of Maybe it would be an idea to write in the naming convention part something about that this is actually meant as the „tag“ to make the connection to docker tag. But in the end that’s a naming problem from docker… |
No no no 😅 The name of an image is the part infront of the The "image name" is |
Okay that at least is what I thought before my confusion started. But then I think the problem for starters is that docker cli itself is confusing in this parts. It returns can not find tag while pushing but actually it can not find the image, because the image does not even exist. Sure also the tag does not exist on that image. And then the command to „rename“ it is called Regarding doc improvement as soon as you know about all this the current docs are no problem. Without that for beginners or people like me which never got in touch with that specific part of docker, it might be not clear what exactly is meant with the naming convention and how to apply it. Clearly you could quickly google how to use external registry with docker and stackoverflow will give you the answer, but I think it is worth adding 3 lines to the docs to reduce that need. |
Perhaps a disclaimer that the hostname portion is required? And that otherwise docker will default to use docker hub? |
@KN4CK3R Maybe like this? |
855d54e
to
f247fe0
Compare
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
I was unable to create a backport for 1.21. @wienans, please send one manually. 🍵
|
Do we need a backport for the docs? |
Yes, as they are versioned. |
* giteaofficial/main: (23 commits) Remove jQuery from SSH key form parser (go-gitea#29193) Refactor request function (go-gitea#29187) Docker Tag Information in Docs (go-gitea#29047) Fix gitea-action user avatar broken on edited menu (go-gitea#29190) Disable parallel Make execution (go-gitea#29186) Auto-update the system status in admin dashboard (go-gitea#29163) Avoid vue warning in dev mode (go-gitea#29188) Update JS and PY dependencies (go-gitea#29184) [skip ci] Updated translations via Crowdin Implement contributors graph (go-gitea#27882) Add support for action artifact serve direct (go-gitea#29120) Advertise WebAuthn support (go-gitea#29176) Tweak repo header (go-gitea#29134) Change webhook-type in create-view (go-gitea#29114) Remove jQuery from the comment task list (go-gitea#29170) Fix can not select team reviewers when reviewers is empty (go-gitea#29174) move sign in labels to be above inputs (go-gitea#28753) Refactor locale&string&template related code (go-gitea#29165) Extract linguist code to method (go-gitea#29168) bump to use go 1.22 (go-gitea#29119) ...
Add more details for the docker tag when using container registry. --------- Co-authored-by: wxiaoguang <wxiaoguang@gmail.com> Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
Add more details for the docker tag when using container registry. --------- Co-authored-by: wxiaoguang <wxiaoguang@gmail.com> Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
Backport #29047 Add more details for the docker tag when using container registry. Co-authored-by: wienans <40465543+wienans@users.noreply.github.com> Co-authored-by: wxiaoguang <wxiaoguang@gmail.com> Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
docker tag
command to the docs for container registry.For Context i was using a own container registry the first time and also didn't pushed any docker images to docker.io so i wasn't aware of the necessity of the docker tag command. i only tried to follow the instructions in the page and got the error:
"An image does not exist locally with the tag: {domain}/{owner}/{image}:{tag}"